-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
【グリッドカラムカード 】ツールバーからのリンク指定機能にnoreferrer、nofollow、link descriptionオプションを追加 #2427
【グリッドカラムカード 】ツールバーからのリンク指定機能にnoreferrer、nofollow、link descriptionオプションを追加 #2427
Conversation
9366c3c
to
f1baa8a
Compare
@akito-38 気になる点が
@kurudrive @goutetsuguma @mtdkei どなたか下記にご意見いただけると助かります
|
確認待ちに戻しました 念のため終礼でも共有してます |
@sysbird |
@sysbird aタグの中のscreen-reader-textについては消すのは簡単に対応できます。 |
@mtdkei |
@akito-38 後でよいので下記対応していただけるとよいかと〜
|
@sysbird |
調整ありがとうございます!以下を確認しました。
マージします |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@akito-38 すみません、以下ご確認いただけますと嬉しいです
readme.txt
Outdated
@@ -108,7 +108,7 @@ e.g. | |||
|
|||
== Changelog == | |||
|
|||
[ Add function ][ Icon / Slider ] Added noreferrer, nofollow, and link description options to the link feature. | |||
[ Add function ][ Icon / Slider ] Added noreferrer and nofollow to the link feature. | |||
[ Add function ][ Grid Column Card (Pro) ] Added noreferrer, nofollow, and link description options to the link feature for each card. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@akito-38
すみません、マージしようと思ったのですが、readme.txt を改めてみたところ、今回の修正されたブロックはグリッドカードブロックのようなので、「readme の and link description options を削除」するのは、112行目の
[ Add function ][ Grid Column Card (Pro) ] Added noreferrer, nofollow, and link description options to the link feature for each card.
の行の「and link description options 」なのかなと思いましたがいかがでしょうか。
お手数ですが、ご確認いただけますと幸いです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@goutetsuguma
すみません。ご指摘ありがとうございます。
修正しましたので、確認お願いいたします。
ありがとうございます。確認しました、マージします |
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
#2199
どういう変更をしたか?
リンクツールバーに Edit link を追加し、a タグの rel やアクセシビリティ対応のテキストを設定できるようにしました。
・Add noreferrerと Add nofollow: rel に noreferrer と nofollow をそれぞれ設定できます。外すことも可能です。
・Accesibility link description: アクセシビリティ対応のテキスト設定にテキストを入れると、spanにテキストが入ります。デフォルトでは 「ブロック名 + link」のテキストが設定されます。
スクリーンショットまたは動画
変更前 Before
変更後 After
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
プログラムの変更の場合
変更内容について何を確認したか、どういう方法で確認をしたかなど
aria-label属性が削除されていることを確認しました。
ツールバーおよび、左側のサイドバーで「リンクを別ウィンドウで開く」のチェックがない場合、targetの出力がないことをそれぞれ確認しました。
ツールバーおよび、左側のサイドバーで「リンクを別ウィンドウで開く」のチェックがある場合、targetが出力され、「_blank」が入ることをそれぞれ確認しました。
ツールバーおよび、左側のサイドバーで「noreferrer を追加」「nofollow を追加」のチェックがない場合、relの出力がないことをそれぞれ確認しました。
ツールバーおよび、左側のサイドバーで「noreferrer を追加」「nofollow を追加」のチェックがある場合、relが出力され、「noreferrer」「nofollow」がそれぞれ入ることを確認しました。
ツールバーおよび、左側のサイドバーで「リンクの説明」が空欄の場合、「screen-reader-text」クラスを持つspanタグの中に、「ブロック名 + link」のテキストが出力されている事をそれぞれ確認しました。
ツールバーおよび、左側のサイドバーで「リンクの説明」に適当なテキストを入力し「screen-reader-text」クラスを持つspanタグの中に、入力したテキストが出力されている事をそれぞれ確認しました。
developブランチで作ったリンク設定付きのグリッドカラムカードアイテムブロックがこのブランチに変更してもリカバリーが発生しないことを確認しました。
レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
実装者と同じ
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。